Skip to content

[FIX/#413] Pageable 객체에 ParameterObject 어노테이션 추가 - #414

Open
eeeeeaaan wants to merge 1 commit into
developfrom
fix/#413
Open

[FIX/#413] Pageable 객체에 ParameterObject 어노테이션 추가#414
eeeeeaaan wants to merge 1 commit into
developfrom
fix/#413

Conversation

@eeeeeaaan

@eeeeeaaan eeeeeaaan commented Aug 14, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #413

📝작업 내용

Pageable request에 ParameterObject 어노테이션을 추가하여 인코딩 문제를 해결함.

비고 (Optional)

참고했던 링크 등 참고 사항을 적어주세요. 코드 리뷰하는 사람이 참고해야 하는 내용을 자유로운 형식으로 적을 수 있습니다.

Summary by CodeRabbit

  • 문서화

    • Swagger에서 페이지네이션 조회 API의 Pageable 파라미터가 페이지 번호, 크기, 정렬 등의 항목으로 명확하게 표시됩니다.
    • 백오피스, 제휴, 리뷰, 학생 관련 목록 조회 API의 API 문서 가독성과 사용성이 개선되었습니다.

  • 기타

    • 로컬 설정 파일과 부하 테스트 디렉터리가 버전 관리 대상에서 제외됩니다.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 43ef52eb-c02a-4174-b476-6554f610fe60

📥 Commits

Reviewing files that changed from the base of the PR and between 3f30ed0 and cd60ff6.

📒 Files selected for processing (7)
  • .gitignore
  • src/main/java/com/assu/server/domain/backoffice/controller/BackofficeMemberController.java
  • src/main/java/com/assu/server/domain/backoffice/controller/BackofficePaperController.java
  • src/main/java/com/assu/server/domain/backoffice/controller/BackofficePartnershipController.java
  • src/main/java/com/assu/server/domain/partnership/controller/PartnershipController.java
  • src/main/java/com/assu/server/domain/review/controller/ReviewController.java
  • src/main/java/com/assu/server/domain/student/controller/StudentController.java
📝 Walkthrough

Walkthrough

조회 API의 Pageable 파라미터에 @ParameterObject를 추가했습니다. Swagger의 sort 값 처리 오류를 수정했습니다. .gitignore에 CLAUDE.md와 /loadtest를 추가했습니다.

Changes

Swagger Pageable 파라미터

Layer / File(s) Summary
조회 API Pageable 문서화
src/main/java/com/assu/server/domain/*/controller/*.java		 백오피스, 제휴, 리뷰, 학생 조회 API의 Pageable 파라미터에 @ParameterObject를 추가했습니다. 관련 import도 추가했습니다.

저장소 무시 규칙

Layer / File(s) Summary
개발 파일 무시 규칙
.gitignore		 CLAUDE.md와 루트 /loadtest 디렉터리를 무시하도록 변경했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to cd60f

The PR adds Pageable parameter annotations to the affected controllers to address request encoding behavior. No actionable merge-blocking risk remains beyond normal checks and review.

Poem

Pageable이 길을 찾고
sort는 배열을 벗어나며
Swagger 문서가 반듯해집니다.
무시 목록에는 새 규칙이 톡,
작은 수정이 큰 오류를 막습니다.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning .gitignore에 CLAUDE.md와 /loadtest를 추가한 변경은 이슈 #413의 Pageable 수정 범위와 관련이 없습니다. .gitignore 변경을 별도 PR로 분리하거나, 이 PR에서 해당 변경을 제거하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 Swagger의 Pageable sort 인코딩 문제를 해결하기 위한 @ParameterObject 추가 변경을 명확하게 설명합니다.
Linked Issues check ✅ Passed 여러 컨트롤러의 Pageable 파라미터에 @ParameterObject를 추가하여 이슈 #413의 수정 요구사항을 충족합니다.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#413

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FIX/#413] sort 값 인코딩 문제

1 participant

@eeeeeaaan